-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add icarAttentionEventResource issue 393 #407
Conversation
Add icarAttentionEventResource and associated enumerations. TODO: Add to Example URL Scheme. Resolves adewg#393
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks ok for me
Add an optional `severity` field that has the same values as that used in disease diagnosis.
I think this PR is now ready to go. It includes the changes from the recent technical group discussions, and now has APIs in exampleURLScheme and healthURLScheme OpenAPI specifications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not use this one anymore, since we have the severity index now.
My earlier suggestion to include "Critical" here might be irrelevant. I suggest leaving "critical" out again, but it is not a deal breaker.
Add icarAttentionEventResource and associated enumerations.
TODO: Add to Example URL Scheme.
Resolves #393